Skip to content

fix: use create-pull-request action for leaderboard updates - #642

Merged
Harxhit merged 1 commit into
Dev-Card:mainfrom
ramnnn2006:fix/641-leaderboard-workflow-protected-branch
Jun 24, 2026
Merged

fix: use create-pull-request action for leaderboard updates#642
Harxhit merged 1 commit into
Dev-Card:mainfrom
ramnnn2006:fix/641-leaderboard-workflow-protected-branch

Conversation

@ramnnn2006

@ramnnn2006 ramnnn2006 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Summary

The contributor leaderboard workflow currently fails because it tries to push updates directly to the main branch, which is protected. This pull request modifies the workflow to use the peter-evans/create-pull-request action. Instead of pushing to main, the workflow will now open an automated pull request with the updated leaderboard data.

Closes #641

Type of Change

  • Bug fix
  • New feature
  • Refactor (no functional change)
  • UI / Design change
  • Tests only
  • Documentation
  • Infrastructure / DevOps
  • Security

What Changed

  • Changed the pull-requests permission in .github/workflows/leaderboard.yml from read to write to allow the action to open pull requests.
  • Replaced the manual git commit and push steps in .github/workflows/leaderboard.yml with the peter-evans/create-pull-request action.

How to Test

  1. Verify the changes by running the workflow in a repository with protected branches.
  2. The workflow should run the script, find the modified leaderboard.json file, and open a pull request without any push block errors.

Checklist

  • My code follows the project's coding style (npm run lint passes).
  • TypeScript compiles without errors (npm run typecheck --workspaces --if-present).
  • I have added or updated tests for the changes I made.
  • All tests pass locally (npm run test --workspaces --if-present).
  • I have updated documentation where necessary.
  • No new console.log or debug statements left in the code.
  • Breaking changes are documented in this PR description.

Copilot AI review requested due to automatic review settings June 23, 2026 09:29
@vercel

vercel Bot commented Jun 23, 2026

Copy link
Copy Markdown

@ramnnn2006 is attempting to deploy a commit to the Prashantkumar Khatri's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI reviewed Jun 23, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions github-actions Bot added devops gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking. labels Jun 23, 2026
@github-actions

github-actions Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Hi @ramnnn2006,

Thanks for opening this pull request.

This PR has been automatically classified based on the files modified.

Applied Labels

  • gssoc:approved
  • devops

Primary Review Area

  • devops

Reviewer

@ShantKhatri has been identified as the primary reviewer for this pull request.

If you have any questions regarding the affected area or implementation details, feel free to reach out to the assigned reviewer.

Thank you for your contribution!

@github-actions

github-actions Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

CI — All Checks Passed

Backend — SKIP

Check Result
Lint -
Test -
Typecheck -

Mobile — SKIP

Check Result
Lint -
Test -

Web — SKIP

Check Result
Build -

Last updated: Tue, 23 Jun 2026 09:29:47 GMT

Harxhit
Harxhit approved these changes Jun 24, 2026
View reviewed changes

@Harxhit Harxhit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Approving

@Harxhit
Harxhit merged commit d1aa2c8 into Dev-Card:main Jun 24, 2026
7 of 8 checks passed
@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Congratulations @ramnnn2006 on getting PR #642 merged!

Thank you for your contribution to the project.

To receive the appropriate GSSoC labels and recognition, please mention @Harxhit in the #get-labels channel on our Discord server and share your merged PR link.

ShantKhatri
ShantKhatri reviewed Jun 24, 2026
View reviewed changes
Comment thread .github/workflows/leaderboard.yml
@Harxhit Harxhit added level:advanced Complex contribution involving deeper technical work. (+55 pts) quality:clean PR is well-structured, readable, and follows good practices. (×1.2 multiplier) type:performance Performance optimization (+15 pts) type:refactor Code refactoring/cleanup (+10 pts) type:devops CI/CD, infra, deployment, workflow work (+15 pts) type:bug Bug fixes (+10 pts) labels Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ShantKhatri ShantKhatri ShantKhatri left review comments
Copilot code review Copilot Copilot left review comments
@Harxhit Harxhit Harxhit approved these changes

Assignees

No one assigned

Labels

devops gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking. level:advanced Complex contribution involving deeper technical work. (+55 pts) quality:clean PR is well-structured, readable, and follows good practices. (×1.2 multiplier) type:bug Bug fixes (+10 pts) type:devops CI/CD, infra, deployment, workflow work (+15 pts) type:performance Performance optimization (+15 pts) type:refactor Code refactoring/cleanup (+10 pts)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Bug: Leaderboard workflow fails on protected branch repositories

4 participants

@ramnnn2006 @Harxhit @ShantKhatri